From 2ef9b989bff2faf392aedc57902e23739b965cb9 Mon Sep 17 00:00:00 2001 From: Kristian Rietveld Date: Fri, 27 Nov 2009 13:40:30 +0100 Subject: [PATCH] Bug 596473 - Second double-click of GtkTreeView row doesn't emit... Rework double click handling in GtkTreeView. We cannot blindly use the 2BUTTON_PRESS and 3BUTTON_PRESS events. In case a user does two fast double clicks, we receive a 3BUTTON_PRESS and BUTTON_PRESS. We cannot easily deduce two double clicks from this. We have removed the bookkeeping using row references of the last paths clicked. Instead we monitor event time, (x, y) coordinates and compare against double-click-time and double-click-distance ourselves. --- gtk/gtktreeprivate.h | 5 ++- gtk/gtktreeview.c | 93 +++++++++++++++----------------------------- 2 files changed, 34 insertions(+), 64 deletions(-) diff --git a/gtk/gtktreeprivate.h b/gtk/gtktreeprivate.h index f4f1872e80..dd0cf8d2a9 100644 --- a/gtk/gtktreeprivate.h +++ b/gtk/gtktreeprivate.h @@ -143,8 +143,9 @@ struct _GtkTreeViewPrivate gint event_last_x; gint event_last_y; - GtkTreeRowReference *last_button_press; - GtkTreeRowReference *last_button_press_2; + guint last_button_time; + gint last_button_x; + gint last_button_y; GtkRBNode *prelight_node; GtkRBTree *prelight_tree; diff --git a/gtk/gtktreeview.c b/gtk/gtktreeview.c index 0155e7dd2c..47b8e59d81 100644 --- a/gtk/gtktreeview.c +++ b/gtk/gtktreeview.c @@ -1375,6 +1375,9 @@ gtk_tree_view_init (GtkTreeView *tree_view) tree_view->priv->tooltip_column = -1; tree_view->priv->post_validation_flag = FALSE; + + tree_view->priv->last_button_x = -1; + tree_view->priv->last_button_y = -1; } @@ -1613,18 +1616,6 @@ gtk_tree_view_destroy (GtkObject *object) tree_view->priv->drag_dest_row = NULL; } - if (tree_view->priv->last_button_press != NULL) - { - gtk_tree_row_reference_free (tree_view->priv->last_button_press); - tree_view->priv->last_button_press = NULL; - } - - if (tree_view->priv->last_button_press_2 != NULL) - { - gtk_tree_row_reference_free (tree_view->priv->last_button_press_2); - tree_view->priv->last_button_press_2 = NULL; - } - if (tree_view->priv->top_row != NULL) { gtk_tree_row_reference_free (tree_view->priv->top_row); @@ -2790,41 +2781,39 @@ gtk_tree_view_button_press (GtkWidget *widget, } /* Test if a double click happened on the same row. */ - if (event->button == 1) + if (event->button == 1 && event->type == GDK_BUTTON_PRESS) { - /* We also handle triple clicks here, because a user could have done - * a first click and a second double click on different rows. - */ - if ((event->type == GDK_2BUTTON_PRESS - || event->type == GDK_3BUTTON_PRESS) - && tree_view->priv->last_button_press) + if (tree_view->priv->last_button_x != -1) { - GtkTreePath *lsc; + int double_click_time, double_click_distance; - lsc = gtk_tree_row_reference_get_path (tree_view->priv->last_button_press); + g_object_get (gtk_settings_get_default (), + "gtk-double-click-time", &double_click_time, + "gtk-double-click-distance", &double_click_distance, + NULL); - if (lsc) + /* Same conditions as _gdk_event_button_generate */ + if ((event->time < tree_view->priv->last_button_time + double_click_time) && + (ABS (event->x - tree_view->priv->last_button_x) <= double_click_distance) && + (ABS (event->y - tree_view->priv->last_button_y) <= double_click_distance)) { - row_double_click = !gtk_tree_path_compare (lsc, path); - gtk_tree_path_free (lsc); + /* We do no longer compare paths of this row and the + * row clicked previously. We use the double click + * distance to decide whether this is a valid click, + * allowing the mouse to slightly move over another row. + */ + row_double_click = TRUE; } - } - if (row_double_click) - { - if (tree_view->priv->last_button_press) - gtk_tree_row_reference_free (tree_view->priv->last_button_press); - if (tree_view->priv->last_button_press_2) - gtk_tree_row_reference_free (tree_view->priv->last_button_press_2); - tree_view->priv->last_button_press = NULL; - tree_view->priv->last_button_press_2 = NULL; + tree_view->priv->last_button_time = 0; + tree_view->priv->last_button_x = -1; + tree_view->priv->last_button_y = -1; } else { - if (tree_view->priv->last_button_press) - gtk_tree_row_reference_free (tree_view->priv->last_button_press); - tree_view->priv->last_button_press = tree_view->priv->last_button_press_2; - tree_view->priv->last_button_press_2 = gtk_tree_row_reference_new_proxy (G_OBJECT (tree_view), tree_view->priv->model, path); + tree_view->priv->last_button_time = event->time; + tree_view->priv->last_button_x = event->x; + tree_view->priv->last_button_y = event->y; } } @@ -10695,10 +10684,6 @@ gtk_tree_view_set_model (GtkTreeView *tree_view, tree_view->priv->anchor = NULL; gtk_tree_row_reference_free (tree_view->priv->top_row); tree_view->priv->top_row = NULL; - gtk_tree_row_reference_free (tree_view->priv->last_button_press); - tree_view->priv->last_button_press = NULL; - gtk_tree_row_reference_free (tree_view->priv->last_button_press_2); - tree_view->priv->last_button_press_2 = NULL; gtk_tree_row_reference_free (tree_view->priv->scroll_to_path); tree_view->priv->scroll_to_path = NULL; @@ -10710,6 +10695,8 @@ gtk_tree_view_set_model (GtkTreeView *tree_view, tree_view->priv->fixed_height_check = 0; tree_view->priv->fixed_height = -1; tree_view->priv->dy = tree_view->priv->top_row_dy = 0; + tree_view->priv->last_button_x = -1; + tree_view->priv->last_button_y = -1; } tree_view->priv->model = model; @@ -12202,27 +12189,9 @@ gtk_tree_view_real_collapse_row (GtkTreeView *tree_view, gtk_tree_path_free (anchor_path); } - if (gtk_tree_row_reference_valid (tree_view->priv->last_button_press)) - { - GtkTreePath *lsc = gtk_tree_row_reference_get_path (tree_view->priv->last_button_press); - if (gtk_tree_path_is_ancestor (path, lsc)) - { - gtk_tree_row_reference_free (tree_view->priv->last_button_press); - tree_view->priv->last_button_press = NULL; - } - gtk_tree_path_free (lsc); - } - - if (gtk_tree_row_reference_valid (tree_view->priv->last_button_press_2)) - { - GtkTreePath *lsc = gtk_tree_row_reference_get_path (tree_view->priv->last_button_press_2); - if (gtk_tree_path_is_ancestor (path, lsc)) - { - gtk_tree_row_reference_free (tree_view->priv->last_button_press_2); - tree_view->priv->last_button_press_2 = NULL; - } - gtk_tree_path_free (lsc); - } + /* Stop a pending double click */ + tree_view->priv->last_button_x = -1; + tree_view->priv->last_button_y = -1; remove_expand_collapse_timeout (tree_view); -- 2.30.2